-
Notifications
You must be signed in to change notification settings - Fork 50
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
The pronoun "they" can also be singular in English #37
base: main
Are you sure you want to change the base?
Conversation
|
Not yet.
Probably
Not yet known; it's a proof of concept. |
Adding in ambiguity usually means having to do something with it, otherwise it will likely make things worse. I have no disagreement with the principle, but this requires more thought than just adding an analysis. |
Yes, as with most commits, this needs additional work downstream. |
I encourage you to do them and attach them to this PR. |
I am not a dev for most of the pairs this will affect. Community buy-in here is a must. |
No-one is a dev for most of the pairs with English. |
According to @mr-martian, pairs with nor, fin, gle, hye, kmr, and gl mention 3rd person pronouns in their dix files, but don't hardcode |
None of these are production-level pairs. |
At least @marcriera should take a look, who actively contributes to |
I'm for the change, but I'd rather add it with Besides the disambiguation aspects, this requires transfer modifications in each pair to recognise the singular pronoun and translate it accordingly (in English-Catalan, where explicit pronouns are usually removed, it should be simple). "They/them/their/theirs" also appear in a few paradigms not covered by the initial commit, but there's time to change them once we agree what to do. |
I like the idea of using It's worth noting that in at least some varieties of English (including mine), one place where |
Sorry for the late reply. The most affected pairs (with testvoc errors) would be the ones already mentioned that reference Regarding "themself"/"themselves", there're currently two entries, one for each. The same happens for "yourself"/"yourselves". I don't see any problem there, other than a possible simplification by combining them into a single entry. |
fwiw Fin-eng is not production / stable and can be updated when as necessary, since it was mostly experiments for wmt and wmt doesn't do fin-eng anymore it's even less important in a way. +1 for defaulting to gender-neutral pronouns in lack of evidence otherwise. |
<e r="RL"><p><l>you</l> <r>prpers<s n="prn"/><s n="subj"/><s n="p2"/><s n="m"/><s n="sp"/></r></p></e> | ||
<e r="RL"><p><l>I</l> <r>prpers<s n="prn"/><s n="subj"/><s n="p1"/><s n="f"/><s n="sg"/></r></p></e> | ||
<e r="RL"><p><l>you</l> <r>prpers<s n="prn"/><s n="subj"/><s n="p2"/><s n="f"/><s n="sg"/></r></p></e> | ||
<e r="RL"><p><l>we</l> <r>prpers<s n="prn"/><s n="subj"/><s n="p1"/><s n="f"/><s n="pl"/></r></p></e> | ||
<e r="RL"><p><l>you</l> <r>prpers<s n="prn"/><s n="subj"/><s n="p2"/><s n="f"/><s n="pl"/></r></p></e> | ||
<e r="RL"><p><l>they</l> <r>prpers<s n="prn"/><s n="subj"/><s n="p3"/><s n="f"/><s n="pl"/></r></p></e> | ||
<e r="RL"><p><l>they</l> <r>prpers<s n="prn"/><s n="subj"/><s n="p3"/><s n="f"/><s n="sg"/></r></p></e> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will result in ^prpers<prn><subj><p3><f><sg>$
generating she/they
and similarly for line 881.
No description provided.